Skip to content

Conversation

@ludviggunne
Copy link
Collaborator

No description provided.

@ludviggunne ludviggunne force-pushed the 14290 branch 3 times, most recently from f2aed22 to 775af90 Compare January 31, 2026 16:31
@ludviggunne
Copy link
Collaborator Author

image

@ludviggunne ludviggunne force-pushed the 14290 branch 2 times, most recently from 076e2a0 to d02ffb7 Compare January 31, 2026 18:25
@ludviggunne ludviggunne force-pushed the 14290 branch 2 times, most recently from ac8071b to 4d9f0f5 Compare February 2, 2026 12:10
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 6, 2026

Copy link
Owner

@danmar danmar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is nice! However I might want to embed this in the main window instead. Near the progress bar. I will investigate..

/**
* @bried Details about currently running threads
*/
std::map<int, CheckThread::Details> mThreadDetails;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use QMap


CheckThread::CheckThread(ThreadResult &result) :
mResult(result)
CheckThread::CheckThread(ThreadResult &result, int index)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the parameter name threadIndex would be better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants